chore(manifest): bump YiAgent/OpenCI SHA to c4f0bfa8#154
Conversation
Automated update from on-main-bump-sha workflow. old=9b40a02acafd321f967761716fafcedb4a713f50 new=c4f0bfa8029781b30f641da83d44a0934d7a5e75
ⓘ You've reached your Qodo monthly free-tier limit. Reviews pause until next month — upgrade your plan to continue now, or link your paid account if you already have one. |
📝 WalkthroughWalkthroughThis PR updates the pinned commit SHA for the ChangesYiAgent/OpenCI Dependency Pin Update
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
.github/workflows/reusable-ci.yml (1)
306-306:⚠️ Potential issue | 🟠 Major | ⚡ Quick winUpdate all stale
resolve-opencipins to match manifest.yml.Line 306 in
reusable-ci.ymland four additional pins inreusable-self-test.yml(lines 73, 190, 213, 258) still point to9bd9cf..., which is inconsistent with the manifest pinc4f0bfa.... This causes SHA consistency failures and breaks pin uniformity across the repository.Verification results showing all affected locations
manifest.yml YiAgent/OpenCI SHA: c4f0bfa8029781b30f641da83d44a0934d7a5e75 MISMATCH .github/workflows/reusable-self-test.yml:Line 73 => 9bd9cf085ba9d3a199b701d42ea054e1625a65ce MISMATCH .github/workflows/reusable-self-test.yml:Line 190 => 9bd9cf085ba9d3a199b701d42ea054e1625a65ce MISMATCH .github/workflows/reusable-self-test.yml:Line 213 => 9bd9cf085ba9d3a199b701d42ea054e1625a65ce MISMATCH .github/workflows/reusable-self-test.yml:Line 258 => 9bd9cf085ba9d3a199b701d42ea054e1625a65ce MISMATCH .github/workflows/reusable-ci.yml:Line 306 => 9bd9cf085ba9d3a199b701d42ea054e1625a65ceUpdate all five occurrences to
c4f0bfa8029781b30f641da83d44a0934d7a5e75.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.github/workflows/reusable-ci.yml at line 306, Replace the stale pin for the action reference "YiAgent/OpenCI/actions/_common/resolve-openci@9bd9cf085ba9d3a199b701d42ea054e1625a65ce" with the manifest pin "c4f0bfa8029781b30f641da83d44a0934d7a5e75" in every workflow occurrence (there are five instances); search for the exact action string "YiAgent/OpenCI/actions/_common/resolve-openci@" to find each spot and update the trailing SHA only so all references match the manifest.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In @.github/workflows/reusable-ci.yml:
- Line 306: Replace the stale pin for the action reference
"YiAgent/OpenCI/actions/_common/resolve-openci@9bd9cf085ba9d3a199b701d42ea054e1625a65ce"
with the manifest pin "c4f0bfa8029781b30f641da83d44a0934d7a5e75" in every
workflow occurrence (there are five instances); search for the exact action
string "YiAgent/OpenCI/actions/_common/resolve-openci@" to find each spot and
update the trailing SHA only so all references match the manifest.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: 24684439-9e8d-41d1-84e4-45e23db39319
📒 Files selected for processing (12)
.github/workflows/agent.yml.github/workflows/ci-self-test.yml.github/workflows/ci.yml.github/workflows/dependencies.yml.github/workflows/docs.yml.github/workflows/issue-ops.yml.github/workflows/on-maintenance.yml.github/workflows/pull-request.yml.github/workflows/release.yml.github/workflows/reusable-ci.yml.github/workflows/reusable-pr.ymlmanifest.yml
|
Superseded by |
… its path claude-code-action@v1.x exposes only execution_file (a path to a JSONL transcript); claude-harness re-exports it as the deprecated `result` alias. Five pre-v1 callers (flag-audit, agent-test, agent-test-gen, create-release, eval-smoke) consumed `result` as if it were Claude's text output, so they emitted the raw path. The visible symptom was flag-audit filing issues whose body was literally "/home/runner/_work/_temp/claude-execution-output.json" (#177). Add a shared _common/extract-text action that reads the execution file and recovers the agent's final text (result record -> last assistant text blocks -> single-object .result -> plain-text passthrough), and wire it into all five callers. Keep the `result` alias for any unmigrated consumers. Closes #177



Automated SHA bump:
9b40a02atoc4f0bfa8The YiAgent/OpenCI self-reference SHA in manifest.yml was stale. Updated to the latest main HEAD so all reusable workflow calls resolve correctly.
Need help on this PR? Tag
@codesmithwith what you need.Summary by CodeRabbit